-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: custom configuration of crio in testing cluster #542
Conversation
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/test e2e-gcp-common-templates-centos7 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test e2e-gcp-common-templates-centos7 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
automation/test.sh
Outdated
@@ -70,11 +70,12 @@ function latest_version() { | |||
export KUBEVIRT_VERSION=$(latest_version "kubevirt") | |||
|
|||
# Latest released CDI version | |||
export CDI_VERSION=$(latest_version "containerized-data-importer") | |||
#export CDI_VERSION=$(latest_version "containerized-data-importer") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's no longer required, we should drop this line to keep the file clean.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just testing PR, which should investigate where is the problem in CI. This change will not be part of result PR
/test e2e-gcp-common-templates-centos7 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test e2e-gcp-common-templates-centos7 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
ad8c933
to
da0381f
Compare
/test e2e-gcp-common-templates-centos7 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
our e2e are not working due to permission issue in crio. cri-o/cri-o#7192 should revert the change. Until that revert PR is released and available in ci envs, we have to set special configuration on crio on all nodes Signed-off-by: Karel Simon <ksimon@redhat.com>
/test e2e-gcp-common-templates-centos-stream9 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test e2e-gcp-common-templates-rhel9 |
@ksimon1: No presubmit jobs available for kubevirt/common-templates@master In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
d1bcf99
to
ad3e476
Compare
/retest |
Set requests and limits for CDI all these changes should help with solving issue, when import of datavolume is not working Signed-off-by: Karel Simon <ksimon@redhat.com>
@ksimon1: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
This PR is no longer needed |
What this PR does / why we need it:
our e2e are not working due to permission issue in crio.
cri-o/cri-o#7192 should revert the change.
Until that revert PR is released and available in ci envs, we have to
set special configuration on crio on all nodes
Release note: